Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optionally enable Spinnaker feature flags #12

Merged

Conversation

dpujadas
Copy link

Enable a list of feature flags as explained in the docs (https://www.spinnaker.io/reference/halyard/commands/#hal-config-features-edit)

@viglesiasce
Copy link
Owner

@dpujadas can you take a quick look at what I added recently here:
https://github.com/viglesiasce/charts/blob/spin-v1.0.0/stable/spinnaker/README.md#automated

I'm trying to avoid having to add a values flag for all of the supported customizations. WDYT?

@dpujadas
Copy link
Author

I think that your approach with automated commands is well suited for:

  • hal commands with lots of parameters / options (as you said, it would be hard to have values for ALL of the supported customizations)
  • future hal features not supported by a specific chart version

In my opinion, enabling feature flags is easier to handle with a yaml list. Do you agree?

@viglesiasce
Copy link
Owner

Hey @dpujadas!

I'm partially convinced although a lot of those features require significant out of band configuration to be useful (artifacts, chaos, and mine-canary). That said I think we should enable jobs by default for the chart.

Can you make the change to default to jobs being enabled?

@dpujadas
Copy link
Author

'Jobs' enabled by default

@viglesiasce
Copy link
Owner

Thanks @dpujadas!

@viglesiasce viglesiasce merged commit 267801f into viglesiasce:spin-v1.0.0 Jul 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants